Conversation
Nextflow's v2 syntax parser requires ConfigScope implementations to have a no-arg constructor so it can reflectively instantiate them at parse time to discover and validate config options. Without this, the co2footprint scope and its sub-scopes are not registered, causing "Unrecognized config option" warnings for all co2footprint.* settings. This adds no-arg constructors to CO2FootprintConfig and all BaseFileConfig subclasses (TraceFileConfig, SummaryFileConfig, ReportFileConfig, DataFileConfig), following the same pattern used by nf-schema's ValidationConfig. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Jonathan Manning <jonathan.manning@seqera.io>
Per Ben's review, only the top-level scope needs @ScopeName, a no-arg constructor, and extension point registration. Nested scopes are discovered via fields on the parent class. - Register CO2FootprintConfig in build.gradle extensionPoints - Remove @ScopeName from nested scope classes (Trace/Summary/Report/DataFileConfig) - Remove no-arg constructors from nested scopes and BaseFileConfig - Remove unnecessary types=[...] from @ConfigOption annotations (kept types=[Number, BigDecimal] on ci since it constructs CiRecord from a number) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Jonathan Manning <jonathan.manning@seqera.io>
Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
…separation) Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
…c values in report Documentation: Added changed to CHANGELOG.md Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
Refactor: Changed default of `emissionMetricsOnly` to `true` Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
Test: Adjusted tests and file checks Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
Refactor: Renamed emission metrics to camel case Test: Adjusted tests to new naming Chore: Intermediate save Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
Refactor: Renamed emission metrics to camel case Test: Adjusted tests to new naming Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
Co-authored-by: Nadja Volkmann <96422527+nadnein@users.noreply.github.com>
Co-authored-by: Nadja Volkmann <96422527+nadnein@users.noreply.github.com>
Fix: Improve report quality for extension / CLI runs
# Conflicts: # src/testResources/cli/data_test.yaml # src/testResources/cli/file_checks.json # src/testResources/cli/report_test.html
Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
Refactor: Move to provenance file and update related tests
…org/bioschemas.org classes annotations to enabled machine processing Test: Adjusted file checks Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
Documentation: Included changed into CHANGELOG.md Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
…t plugin version Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
# Conflicts: # CHANGELOG.md # src/testResources/cli/file_checks.json # src/testResources/cli/report_test.html # src/testResources/observer/file_checks.json # src/testResources/report/test_payload.json
Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
Signed-off-by: nadnein <nadjav.18@gmail.com>
Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
Signed-off-by: nadnein <nadjav.18@gmail.com>
Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
Signed-off-by: nadnein <nadjav.18@gmail.com>
Signed-off-by: nadnein <nadjav.18@gmail.com>
Fix: zero division crashes when task realtime is 0
Fix floating point error and refactor Bytes for value types
Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
Add --delimiter argument to CLI post-run docs
JosuaCarl
added
the
🛰️ continuous integration
nadnein
approved these changes
May 4, 2026
Chore: Version 1.3.0 bump
…or-2 Fix/config scope noarg constructor 2
Resolves conflicts in:
- src/main/nextflow/co2footprint/Config/BaseFileConfig.groovy
(drop @ConfigOption imports; the annotations live on each subclass now)
- src/main/nextflow/co2footprint/Logging/CustomCaptureAppender.groovy
(take dev's 26.04.0 LogObserver adaptation)
- src/testResources/{cli,observer,report}/* (take dev's calculator outputs)
- Mark file/enabled/overwrite/maxTasks final on the four nested file scopes and final on name/ending/defaultEnabled in BaseFileConfig (each is assigned exactly once in a constructor body) - Normalise whitespace: drop trailing spaces and double-space after `=` - Add CHANGELOG entry under New > Bug Fixes Signed-off-by: Jonathan Manning <jonathan.manning@seqera.io>
…tructor Fix: Add no-arg constructors for v2 config parser scope registration
… file name Test: Adapted file tests Documentation: Added changes to CHANGELOG.md Documentation: Moved fix into this release Signed-off-by: Josua Carl <josua.carl@uni-tuebingen.de>
Fix: CI plot threw error due to changed variable name -> Inserted new…
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Merge
dev->masterfor Release 1.3.0